Skip to content

feat(core): Export spanStreamingIntegration from CDN bundles#20400

Merged
nicohrubec merged 8 commits intodevelopfrom
nh/export-spanstreaming-cdn
Apr 21, 2026
Merged

feat(core): Export spanStreamingIntegration from CDN bundles#20400
nicohrubec merged 8 commits intodevelopfrom
nh/export-spanstreaming-cdn

Conversation

@nicohrubec
Copy link
Copy Markdown
Member

@nicohrubec nicohrubec commented Apr 20, 2026 •
edited
Loading

  • Export the integration from all CDN tracing bundles and shim for non-tracing bundles.
  • Remove the CDN test skips in the browser integration tests since this should work now.
  • Reserve _streamed in the terser mangler so withStreamedSpan's marker survives minification in CDN bundles.
  • Some additional test changes were needed to make CI happy. Mostly cases where certain browsers don't emit some attributes but we check for strict equality with toEqual.

Closes #20273

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 20, 2026 •
edited
Loading

size-limit report 📦

Path Size % Change Change
@sentry/browser 25.88 kB - -
@sentry/browser - with treeshaking flags 24.35 kB - -
@sentry/browser (incl. Tracing) 43.81 kB - -
@sentry/browser (incl. Tracing + Span Streaming) 45.5 kB - -
@sentry/browser (incl. Tracing, Profiling) 48.73 kB - -
@sentry/browser (incl. Tracing, Replay) 82.98 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 72.5 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 87.67 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 99.93 kB - -
@sentry/browser (incl. Feedback) 42.7 kB - -
@sentry/browser (incl. sendFeedback) 30.55 kB - -
@sentry/browser (incl. FeedbackAsync) 35.55 kB - -
@sentry/browser (incl. Metrics) 27.16 kB - -
@sentry/browser (incl. Logs) 27.29 kB - -
@sentry/browser (incl. Metrics & Logs) 27.98 kB - -
@sentry/react 27.62 kB - -
@sentry/react (incl. Tracing) 46.05 kB - -
@sentry/vue 30.71 kB - -
@sentry/vue (incl. Tracing) 45.62 kB - -
@sentry/svelte 25.89 kB - -
CDN Bundle 28.57 kB +0.09% +25 B 🔺
CDN Bundle (incl. Tracing) 46.08 kB +2.54% +1.14 kB 🔺
CDN Bundle (incl. Logs, Metrics) 29.95 kB +0.1% +28 B 🔺
CDN Bundle (incl. Tracing, Logs, Metrics) 47.12 kB +2.37% +1.09 kB 🔺
CDN Bundle (incl. Replay, Logs, Metrics) 68.92 kB +0.05% +28 B 🔺
CDN Bundle (incl. Tracing, Replay) 83.14 kB +1.44% +1.18 kB 🔺
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 84.17 kB +1.38% +1.14 kB 🔺
CDN Bundle (incl. Tracing, Replay, Feedback) 88.61 kB +1.31% +1.14 kB 🔺
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 89.69 kB +1.3% +1.15 kB 🔺
CDN Bundle - uncompressed 83.59 kB +0.23% +186 B 🔺
CDN Bundle (incl. Tracing) - uncompressed 137.62 kB +2.47% +3.31 kB 🔺
CDN Bundle (incl. Logs, Metrics) - uncompressed 87.73 kB +0.22% +186 B 🔺
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 141.03 kB +2.41% +3.31 kB 🔺
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 211.31 kB +0.09% +186 B 🔺
CDN Bundle (incl. Tracing, Replay) - uncompressed 255.06 kB +1.32% +3.31 kB 🔺
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 258.46 kB +1.3% +3.31 kB 🔺
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 267.97 kB +1.26% +3.31 kB 🔺
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 271.36 kB +1.24% +3.31 kB 🔺
@sentry/nextjs (client) 48.58 kB - -
@sentry/sveltekit (client) 44.22 kB - -
@sentry/node-core 58.02 kB +0.02% +7 B 🔺
@sentry/node 174.89 kB +0.01% +6 B 🔺
@sentry/node - without tracing 97.98 kB +0.02% +10 B 🔺
@sentry/aws-serverless 115.21 kB +0.02% +18 B 🔺

View base workflow run

@nicohrubec nicohrubec requested a review from chargome April 20, 2026 15:25
@nicohrubec nicohrubec marked this pull request as ready for review April 20, 2026 15:26
chargome
chargome approved these changes Apr 21, 2026
View reviewed changes
Comment thread packages/integration-shims/src/SpanStreaming.ts Outdated
export const spanStreamingIntegrationShim = defineIntegration(() => {
consoleSandbox(() => {
// eslint-disable-next-line no-console
console.warn('You are using spanStreamingIntegration() even though this bundle does not include span streaming.');
Copy link
Copy Markdown
Member

@chargome chargome Apr 21, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
console.warn('You are using spanStreamingIntegration() even though this bundle does not include span streaming.');
console.warn('You are using spanStreamingIntegration() even though this bundle does not include tracing.');

nicohrubec and others added 2 commits April 21, 2026 15:05
@nicohrubec
Merge branch 'develop' into nh/export-spanstreaming-cdn
0cd9233 
Resolve .size-limit.js conflicts in favor of this branch's higher limits.
@nicohrubec @claude
ref(integration-shims): Reword spanStreaming shim warning per review
e00520b 
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@nicohrubec nicohrubec merged commit 29604aa into develop Apr 21, 2026
248 checks passed
@nicohrubec nicohrubec deleted the nh/export-spanstreaming-cdn branch April 21, 2026 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@chargome chargome chargome approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Export spanStreamingIntegration from CDN bundles

2 participants

@nicohrubec @chargome